Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(libs): Show metadata origin #675

Merged
merged 2 commits into from
Nov 8, 2023
Merged

fix(libs): Show metadata origin #675

merged 2 commits into from
Nov 8, 2023

Conversation

Angi-Kinas
Copy link
Collaborator

Now the metadata origin will be displayed if there is data present, even if there is no lineage text:

image

Copy link
Contributor

github-actions bot commented Nov 7, 2023

Affected libs: ui-elements, feature-catalog, feature-record, feature-router, feature-search, feature-map, feature-dataviz, ui-catalog, ui-search,
Affected apps: metadata-editor, datahub, demo, webcomponents, search, map-viewer,

  • 🚀 Build and deploy storybook and demo on GitHub Pages
  • 📦 Build and push affected docker images

@coveralls
Copy link

coveralls commented Nov 7, 2023

Coverage Status

coverage: 86.617% (-0.8%) from 87.443%
when pulling 1f0589c on DH-fix-lineage-display
into d42c29a on main.

metadata.recordUpdated ||
metadata.updateFrequency ||
metadata.status
"
[title]="'record.metadata.origin' | translate"
>
<p class="mb-5 pt-4 whitespace-pre-line break-words" gnUiLinkify>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you need to also hide that block if there's no lineage, otherwise it makes a lot of whitespace

Copy link
Collaborator

@jahow jahow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thanks! let's wait for the backport bot to be operational before merging this :)

@Angi-Kinas Angi-Kinas force-pushed the DH-fix-lineage-display branch from 1f0589c to f4a787c Compare November 8, 2023 10:04
@Angi-Kinas Angi-Kinas merged commit 5f1c197 into main Nov 8, 2023
6 checks passed
@Angi-Kinas Angi-Kinas deleted the DH-fix-lineage-display branch November 8, 2023 10:27
Copy link
Contributor

github-actions bot commented Nov 8, 2023

Error on backporting to branch 2.0.x, see Docker Hub actions for more information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants